Skip to content

Fix the regridding error when the target latitude or longitude is exactly 0.0#385

Merged
yantosca merged 2 commits intogeoschem:mainfrom
1Dandan:regrid-stretched
Feb 2, 2026
Merged

Fix the regridding error when the target latitude or longitude is exactly 0.0#385
yantosca merged 2 commits intogeoschem:mainfrom
1Dandan:regrid-stretched

Conversation

@1Dandan
Copy link
Contributor

@1Dandan 1Dandan commented Oct 27, 2025

Name and Institution (Required)

Name: Dandan Zhang
Institution: Harvard University

Describe the update

To fix the regridding error when the target latitude or longitude is exactly 0.0

Expected changes

It will work for stretched GCHP regridding when the target latitude or longitude is exactly 0.0.

Reference(s)

N/A

Related Github Issue

#384

@yantosca yantosca self-requested a review October 28, 2025 14:12
@yantosca yantosca self-assigned this Oct 28, 2025
@yantosca yantosca added topic: Regridding Issues pertaining to horizontal & vertical regridding category: Bug Fix Fixes a bug that was previously reported labels Oct 28, 2025
@yantosca yantosca added this to the 1.7.0 milestone Oct 28, 2025
Copy link
Contributor

@yantosca yantosca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @1Dandan, this looks good but we just need a CHANGELOG.md update.

Copy link
Contributor

@yantosca yantosca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @1Dandan, good to merge!

@yantosca
Copy link
Contributor

NOTE: I forgot to change the base branch. This feature branch has been merged into dev, but the PR will show as merged once we merge dev into main.

@msulprizio msulprizio requested a review from lizziel October 29, 2025 14:19
Copy link
Contributor

@lizziel lizziel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@lizziel
Copy link
Contributor

lizziel commented Jan 21, 2026

@yantosca, is there anything else this PR needs to merge it in?

@yantosca
Copy link
Contributor

@lizziel: It's already merged into dev but I forgot to change the base branch when I did it. It'll show as merged when we merge dev into main.

@yantosca yantosca merged commit 1edd0e4 into geoschem:main Feb 2, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: Bug Fix Fixes a bug that was previously reported topic: Regridding Issues pertaining to horizontal & vertical regridding

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants